-
Notifications
You must be signed in to change notification settings - Fork 81
[ADD] vineyard: Add module for Vineyard #1230
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 19.0
Are you sure you want to change the base?
Conversation
vineyard/data/ir_model_fields.xml
Outdated
| <field name="copied" eval="True"/> | ||
| <field name="field_description">Harvest Location</field> | ||
| <field name="model_id" ref="stock.model_stock_location"/> | ||
| <field name="name">x_harvest_location</field> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| <field name="name">x_harvest_location</field> | |
| <field name="name">x_is_harvest</field> |
50d648f to
b52e6ed
Compare
a7d5cc2 to
14a2bd6
Compare
15c6394 to
6ec9ed6
Compare
d92e062 to
7c8d33c
Compare
2f16b03 to
92797aa
Compare
122fff5 to
60205b5
Compare
60205b5 to
7daa02b
Compare
pgu-odoo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is still in review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clean the dependency. web_studio is already in depends of excise_management
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got warning on runbot: 'web_studio' is missing in the dependencies. (
industry/tests/test_generic/tests/test_xml.py
Line 213 in f17242a
| _logger.warning("'web_studio' is missing in the dependencies.") |
even its depends of excise_management
@vava-odoo
vineyard/data/base_automation.xml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't we move this code to server action to industry_create_lot_on_validation_server_action and have only one cron?
vineyard/data/ir_actions_server.xml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| for ml in picking.move_line_ids.filtered(lambda line: line.product_id and line.product_id.tracking == 'lot' and not line.lot_id): |
can you improve like this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| <field name="model">x_project_task_worksheet_template_1</field> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same model is imported from industry_fsm_report with same name, fields and rules.
So still i need to create model here in this module?
vineyard/data/ir_model_fields.xml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
improve the id please 😃
vineyard/data/ir_rule.xml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| <field name="domain_force"><[]></field> |
will be also fine.
vineyard/data/knowledge_article.xml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| <h1><span class="display-4-fs"><strong>Vineyard</strong></span></h1> |
1deb5d0 to
f17242a
Compare
vineyard/__manifest__.py
Outdated
| 'appointment_hr', | ||
| 'base_industry_data', | ||
| 'excise_management', | ||
| 'hr_sign', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need sign
vineyard/__manifest__.py
Outdated
| 'author': 'Odoo S.A.', | ||
| 'depends': [ | ||
| 'appointment_crm', | ||
| 'appointment_google_calendar', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 'appointment_google_calendar', |
vineyard/__manifest__.py
Outdated
| 'depends': [ | ||
| 'appointment_crm', | ||
| 'appointment_google_calendar', | ||
| 'appointment_hr', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 'appointment_hr', |
| 'industry_fsm_sale_report', | ||
| 'knowledge', | ||
| 'mrp_plm', | ||
| 'pos_enterprise', | ||
| 'product_expiry', | ||
| 'project_purchase', | ||
| 'quality_mrp', | ||
| 'sale_planning', | ||
| 'website_appointment_sale_project', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please clean the dependency
vineyard/data/website_menu.xml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why it is in data ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this 🤔
| <odoo noupdate="1"> | ||
| <record id="product_supplierinfo_3" model="product.supplierinfo"> | ||
| <field name="partner_id" ref="base_industry_data.res_partner_35"/> | ||
| <field name="product_tmpl_id" ref="product_template_31"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this from whole file as you are already defining product_id
vineyard/demo/quality_point.xml
Outdated
| <record id="quality_point_8" model="quality.point"> | ||
| <field name="title">Sensory Evaluation</field> | ||
| <field name="product_category_ids" eval="[(6, 0, [ref('product_category_8')])]"/> | ||
| <field name="note"><
Odoo for VineyardStreamline your entire vineyard operation with this all-in-one management solution. From planting and harvest to wine aging, blending, and final sale, this package ensures total control over your production and team, so you can focus on quality and profitability.
Basics
Task ID: 4965401